-
Notifications
You must be signed in to change notification settings - Fork 75
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add build system and continuous integration (Windows, macOS, Linux) #14
Conversation
I cannot test this script, and im unwilling to merge without testing. When someone test it we should be able to merge without issues! |
Proof of success: https://travis-ci.org/kevinlul/ygopro/builds/547454936 Can also confirm by testing on fresh Ubuntu VM image and following wiki instructions. (This is pretty much Travis though.) Confirmed as working on fresh Windows Subsystem for Linux Ubuntu 18.04 |
At this point, the Linux scripts and CI are working well. I will focus on the remaining platforms and patch the Linux scripts as needed. Platform scripts will be merged into this branch when they are confirmed working well. |
Merged prebuild and deploy system for macOS and updated Travis to match.
Issues:
Possibly remedy for Sierra/High Sierra situation: https://github.com/phracker/MacOSX-SDKs/releases |
Tested on 18.04 Bionic; explanation on wiki fmt no longer needs to be cloned locally; install via apt-get. Moved in premake5.lua so Windows keeps working for now. Added .gitignore for premake5 binary since this script drops it right in the working directory for convenience.
…ng (needs -fms-permissive at least)
- only ignore build/ - all scripts must be run from repo root, see wiki for details - install-ubuntu-apt.sh gets everything you can get from apt - install-ubuntu-src.sh builds fmt and nlohmann-json from source and installs to /usr/local instead of doing some sketchy apt repo shuffling - install-ubuntu-bin.sh downloads premake5 and irrKlang binaries into the working directory
- By default moves the release binaries, set BUILD_CONFIG=debug to choose debug - Only strips the release binary of debug symbols - Does not retrieve FONTS since we can't decide! To do later.
build-essential, cmake, curl are not needed since they are preinstalled TODO: possibly test on other GCC and Clang versions
Corresponding update for ocgcore Current build fails on linking irrlicht: https://pastebin.com/KVL2DTwu
http://irrlicht.sourceforge.net/forum/viewtopic.php?t=36933 Also seen on Koishi's premake
26ef111
to
1c67135
Compare
1c67135
to
ae21376
Compare
babcb75
to
0182f62
Compare
Pending change: revert freetype to system freetype on macOS, test behaviour on Linux and update accordingly. |
Also requires edo9300/ygopro-core#48 |
Removed freetype from vcpkg install Added freetype directories to gitignore
Changes:
To do when or after merging @edo9300 :
|
Tested on 18.04 Bionic; explanation on wiki
fmt no longer needs to be cloned locally; install via apt-get. Moved in premake5.lua so Windows keeps working for now.
Added .gitignore for premake5 binary since this script drops it right in the working directory for convenience.